Skip to content

Conversation

@mokagio
Copy link
Contributor

@mokagio mokagio commented Jan 24, 2023

What it says on the title.

As a confirmation of the successful hotfix merge, notice diff equals to that from #19961 minus the RELEASE-NOTES.txt amendment and that the Version.*.xcconfig files have not changed.

Next steps

I'll ship a new beta for 21.6. Apart from the fact that we can't ship betas from non release branches, I like how the merge commit from this PR will track the hotfix integration in the Git history neatly.

mokagio and others added 5 commits January 24, 2023 11:54
As usual when merging a hotfix into a release branch, we got conflicts
in the `Version.*.xcconfig` files and they have been resolved by keeping
the value from the release branch, as that's what we'll be using for
subsequent builds.
@mokagio mokagio requested a review from a team January 24, 2023 02:10
@mokagio mokagio modified the milestones: 21.5.1, 21.6 ❄️ Jan 24, 2023
@mokagio mokagio enabled auto-merge January 24, 2023 02:10
@wpmobilebot
Copy link
Contributor

You can test the changes in Jetpack from this Pull Request by:
  • Clicking here or scanning the QR code below to access App Center
  • Then installing the build number pr19974-c6f8773 on your iPhone

If you need access to App Center, please ask a maintainer to add you.

@wpmobilebot
Copy link
Contributor

You can test the changes in WordPress from this Pull Request by:
  • Clicking here or scanning the QR code below to access App Center
  • Then installing the build number pr19974-c6f8773 on your iPhone

If you need access to App Center, please ask a maintainer to add you.

Copy link
Contributor

@twstokes twstokes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FWIW it LGTM @mokagio but I understand if we also need to get approval from the Apps Infra team. 👍

@mokagio mokagio merged commit 982fac4 into release/21.6 Jan 24, 2023
@mokagio mokagio deleted the merge/release-21.5.1-into-21.6 branch January 24, 2023 23:58
@mokagio
Copy link
Contributor Author

mokagio commented Jan 25, 2023

but I understand if we also need to get approval from the Apps Infra team. 👍

Auto-merge already kicked in, but anyways I'd say that for a straightforward PR like this one, anyone's approval is fine.

Thank you for always helping @twstokes !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants